Skip to content

[Feat] Header 컴포넌트 수정 및 리팩토링#27

Merged
KyeongJooni merged 14 commits into
mainfrom
24-feat/Header-컴포넌트-수정-및-리팩토링
Jan 18, 2026

Hidden character warning

The head ref may contain hidden characters: "24-feat/Header-\ucef4\ud3ec\ub10c\ud2b8-\uc218\uc815-\ubc0f-\ub9ac\ud329\ud1a0\ub9c1"
Merged

[Feat] Header 컴포넌트 수정 및 리팩토링#27
KyeongJooni merged 14 commits into
mainfrom
24-feat/Header-컴포넌트-수정-및-리팩토링

Conversation

@KyeongJooni

@KyeongJooni KyeongJooni commented Jan 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

✨ 주요 변경사항

  • Header 로그인 상태 및 UserMenu 드롭다운 추가
  • Profile, Card 컴포넌트 리팩토링
  • RadioButton disabled 스타일 수정
  • 컴포넌트 스토리/테스트 추가
  • Storybook GitHub Pages 배포 설정

📝 작업 상세 내용

  • Header: isLoggedIn prop으로 상태 분기, UserMenu 드롭다운
  • Profile: 태그로 변경, Props 네이밍 개선
  • Card: Props 네이밍 간소화, slots 구조 변경
  • RadioButton: disabled 시 회색 dot 표시

✅ 체크리스트

  • PR 본문에 Close #번호 추가
  • 불필요한 주석, 디버깅 코드 제거
  • 기능 테스트 및 정상 동작 확인
  • 커밋컨벤션 / 코드컨벤션 준수

📸 스크린샷 (선택)

image

헤더 컴포넌트

image

라디오 버튼

🔍 기타 참고사항

🔗 관련 이슈

@github-actions github-actions Bot added the ✨Feature 새로운 기능 label Jan 15, 2026
@github-actions

github-actions Bot commented Jan 15, 2026

Copy link
Copy Markdown

PR 검증 결과

✅ TypeScript: 통과
✅ ESLint: 통과
✅ Prettier: 통과
✅ Test: 통과
❌ Build: 실패

@github-actions

github-actions Bot commented Jan 15, 2026

Copy link
Copy Markdown

PR 검증 결과

✅ TypeScript: 통과
✅ ESLint: 통과
✅ Prettier: 통과
✅ Test: 통과
✅ Build: 성공
Build Size: 2MB

@github-actions

github-actions Bot commented Jan 15, 2026

Copy link
Copy Markdown

PR 검증 결과

✅ TypeScript: 통과
✅ ESLint: 통과
✅ Prettier: 통과
✅ Test: 통과
✅ Build: 성공
Build Size: 2MB

@KyeongJooni KyeongJooni added ♻️Refactor 코드 개선 ✅Test 테스트 📝Docs 문서 🛠️ Chore 기타 작업 labels Jan 15, 2026
@vercel

vercel Bot commented Jan 15, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
loopit Ready Ready Preview, Comment Jan 15, 2026 4:17pm

@github-actions

github-actions Bot commented Jan 15, 2026

Copy link
Copy Markdown

PR 검증 결과

✅ TypeScript: 통과
✅ ESLint: 통과
✅ Prettier: 통과
✅ Test: 통과
✅ Build: 성공
Build Size: 측정 불가

@github-actions

github-actions Bot commented Jan 18, 2026

Copy link
Copy Markdown

PR 검증 결과

✅ TypeScript: 통과
⚠️ ESLint: 문제 발견
❌ Prettier: 포맷 필요
❌ Test: 실패
❌ Build: 실패

ESLint 상세

  • Errors: 1
  • Warnings: 0

src/pages/playground.tsx

  • Error (Line 291): Parsing error: '}' expected. unknown

@github-actions

github-actions Bot commented Jan 18, 2026

Copy link
Copy Markdown

PR 검증 결과

✅ TypeScript: 통과
✅ ESLint: 통과
✅ Prettier: 통과
✅ Test: 통과
✅ Build: 성공
Build Size: 3MB

@github-actions

github-actions Bot commented Jan 18, 2026

Copy link
Copy Markdown

PR 검증 결과

✅ TypeScript: 통과
✅ ESLint: 통과
✅ Prettier: 통과
✅ Test: 통과
✅ Build: 성공
Build Size: 3MB

jwj0620gcu
jwj0620gcu approved these changes Jan 18, 2026
View reviewed changes

@jwj0620gcu jwj0620gcu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다! 리팩토링 깔끔하게 해주셔서 감사합니다!

YeBeenChoi
YeBeenChoi approved these changes Jan 18, 2026
View reviewed changes

@YeBeenChoi YeBeenChoi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전체적으로 구조가 많이 정리돼서 코드를 읽기가 훨씬 편해진 것 같아요 👍 Card 컴포넌트에서 props 네이밍 변경하신 것과 Header 컴포넌트에서 variants를 제거하고 cn으로 정리하신 점이 인상적이었습니다! UserMenu로 로직을 분리한 것도 역할이 명확해져서 좋은 것 같아요! 고생 많으셨습니다 👍👍

@KyeongJooni
KyeongJooni merged commit a0703f8 into main Jan 18, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@YeBeenChoi YeBeenChoi YeBeenChoi approved these changes
@jwj0620gcu jwj0620gcu jwj0620gcu approved these changes

Assignees

@KyeongJooni KyeongJooni

Labels

🛠️ Chore 기타 작업 📝Docs 문서 ✨Feature 새로운 기능 ♻️Refactor 코드 개선 ✅Test 테스트

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Feat] #24 - Header 컴포넌트 수정 및 리팩토링

3 participants

@KyeongJooni @YeBeenChoi @jwj0620gcu